Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: hover #220

Merged
merged 1 commit into from
Sep 17, 2023
Merged

feat: hover #220

merged 1 commit into from
Sep 17, 2023

Conversation

mhanberg
Copy link
Collaborator

@mhanberg mhanberg commented Sep 17, 2023

Implements hover for Elixir modules and functions/macros and Erlang functions. Will add support for Erlang modules later.

Converts the internal erlang doc chunk (which is a pseudo html tree structure) into markdown so that they'll be nicely syntax highlighted in text editors that support it.

Some code was adapted from #104, so @sineed is tagged as a co-author on the commit.

Closes #48 #104

TODO

  • type docs
  • erlang module docs

@mhanberg
Copy link
Collaborator Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

Co-authored-by: Denis Tataurov <[email protected]>
@mhanberg mhanberg merged commit c5a68df into main Sep 17, 2023
3 checks passed
@mhanberg mhanberg deleted the 2023-09-14-hover2 branch September 17, 2023 18:47
@sineed
Copy link
Contributor

sineed commented Sep 18, 2023

sorry @mhanberg I started a new job a month ago and didn't have enough of time for the followup changes. I'm glad you found some time for this one 👍

@mhanberg
Copy link
Collaborator Author

No worries at all! Thank you for your efforts.

And congrats! I hope the new job is going well 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

hover docs
2 participants